Skip to content

fix: use pull_request.user.login instead of github.actor for bot check#2050

Merged
adamtheturtle merged 1 commit intomainfrom
fix/dependabot-actor-check
Mar 17, 2026
Merged

fix: use pull_request.user.login instead of github.actor for bot check#2050
adamtheturtle merged 1 commit intomainfrom
fix/dependabot-actor-check

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Mar 17, 2026

Fixes adamtheturtle/literalizer#146

The github.actor check is unreliable - it reflects who triggered the workflow, not who opened the PR. Use github.event.pull_request.user.login instead.

Flagged by zizmor bot-conditions audit.

Made with Cursor


Note

Low Risk
Small CI workflow condition tweak plus a static-analysis config update; risk is limited to potentially changing when Dependabot auto-merge runs.

Overview
Fixes the Dependabot auto-merge workflow to gate on the PR author (via github.event.pull_request.user.login) instead of the triggering actor, making bot detection reliable.

Updates zizmor.yml by removing the bot-conditions rule disable now that the workflow condition is compliant.

Written by Cursor Bugbot for commit 495367b. This will update automatically on new commits. Configure here.

@adamtheturtle adamtheturtle force-pushed the fix/dependabot-actor-check branch from db5c974 to 495367b Compare March 17, 2026 09:15
@adamtheturtle adamtheturtle merged commit 53ab009 into main Mar 17, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the fix/dependabot-actor-check branch March 17, 2026 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dependabot-merge.yml: use pull_request.user.login instead of github.actor for bot check

1 participant